- 
                Notifications
    You must be signed in to change notification settings 
- Fork 252
Improvement/cldsrv 724 bucket related functional tests #5976
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improvement/cldsrv 724 bucket related functional tests #5976
Conversation
        
          
                tests/functional/aws-node-sdk/test/bucket/aclUsingPredefinedGroups.js
              
                Outdated
          
            Show resolved
            Hide resolved
        
              
          
                tests/functional/aws-node-sdk/test/bucket/aclUsingPredefinedGroups.js
              
                Outdated
          
            Show resolved
            Hide resolved
        
              
          
                tests/functional/aws-node-sdk/test/bucket/aclUsingPredefinedGroups.js
              
                Outdated
          
            Show resolved
            Hide resolved
        
              
          
                tests/functional/aws-node-sdk/test/bucket/bucketPolicyWithResourceStatements.js
              
                Outdated
          
            Show resolved
            Hide resolved
        
              
          
                tests/functional/aws-node-sdk/test/bucket/bucketPolicyWithResourceStatements.js
              
                Outdated
          
            Show resolved
            Hide resolved
        
              
          
                tests/functional/aws-node-sdk/test/bucket/putBucketReplication.js
              
                Outdated
          
            Show resolved
            Hide resolved
        
              
          
                tests/functional/aws-node-sdk/test/bucket/putBucketReplication.js
              
                Outdated
          
            Show resolved
            Hide resolved
        
              
          
                tests/functional/aws-node-sdk/test/bucket/putBucketReplication.js
              
                Outdated
          
            Show resolved
            Hide resolved
        
              
          
                tests/functional/aws-node-sdk/test/bucket/putBucketObjectLock.js
              
                Outdated
          
            Show resolved
            Hide resolved
        
              
          
                tests/functional/aws-node-sdk/test/bucket/putBucketObjectLock.js
              
                Outdated
          
            Show resolved
            Hide resolved
        
      | after(async () => { | ||
| await s3ObjectLock.send(new DeleteBucketCommand({ Bucket: bucket })); | ||
| }); | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| after(async () => { | |
| await s3ObjectLock.send(new DeleteBucketCommand({ Bucket: bucket })); | |
| }); | |
| after(() => s3ObjectLock.send(new DeleteBucketCommand({ Bucket: bucket }))); | 
        
          
                tests/functional/aws-node-sdk/test/bucket/testBucketVersioning.js
              
                Outdated
          
            Show resolved
            Hide resolved
        
      | // Ensure host includes port for canonical request | ||
| const hostname = host.split(':')[0]; // Extract 127.0.0.1 | ||
| const port = parseInt(host.split(':')[1] || '8000', 10); // Default to 8000 | ||
| const [pathBase, queryString] = path.split('?'); | ||
| const query = queryString ? Object.fromEntries(new URLSearchParams(queryString)) : {}; | ||
|  | ||
| // Create HTTP request (mimics AWS.HttpRequest with v2-like endpoint structure) | ||
| const request = new HttpRequest({ | ||
| protocol: 'http:', // Match Scality CloudServer | ||
| hostname, // 127.0.0.1 | ||
| port, // 8000 | ||
| method: method.toUpperCase(), | ||
| path: pathBase, | ||
| query, | ||
| body, | ||
| headers: { | ||
| Host: host, // Explicitly set Host: 127.0.0.1:8000 | ||
| 'X-Amz-Date': signingDate.toISOString().replace(/[:\-]|\.\d{3}/g, ''), | ||
| }, | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
comments to be removed probably
| ❌ 119 Tests Failed:
 View the top 3 failed test(s) by shortest run time
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 View the full list of 92 ❄️ flaky test(s)
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 To view more test analytics, go to the Test Analytics Dashboard | 
69b2a4a    to
    c56ac68      
    Compare
  
    f6cbcbd    to
    cc2320a      
    Compare
  
    
No description provided.